<html>
<head><meta charset="utf-8"><title>rustdoc bisection #61560 · t-compiler · Zulip Chat Archive</title></head>
<h2>Stream: <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/index.html">t-compiler</a></h2>
<h3>Topic: <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html">rustdoc bisection #61560</a></h3>

<hr>

<base href="https://rust-lang.zulipchat.com">

<head><link href="https://rust-lang.github.io/zulip_archive/style.css" rel="stylesheet"></head>

<a name="177326511"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177326511" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> lqd <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177326511">(Oct 04 2019 at 10:22)</a>:</h4>
<p>hey <span class="user-mention" data-user-id="116083">@pnkfelix</span> , re <a href="https://github.com/rust-lang/rust/issues/61560#issuecomment-538340224" target="_blank" title="https://github.com/rust-lang/rust/issues/61560#issuecomment-538340224">this</a>, do you want me to extract the doctests and bisect their code to see the commits which do change resolution wrt extern crate and the likes ?<br>
the TLDR of this issue is: there were some rustc bugs which seem fixed, there are invalid tests, there were some rustdoc changes which started detecting these invalid tests, and there might still be a miscommunication between rustdoc and rustc wrt the soft_unstable lint</p>



<a name="177326622"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177326622" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> lqd <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177326622">(Oct 04 2019 at 10:23)</a>:</h4>
<p>since they're now fixed I don't think we really need to, but I'll extract the soft_unstable piece of code and post it on the tracking issue</p>



<a name="177326977"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177326977" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> lqd <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177326977">(Oct 04 2019 at 10:29)</a>:</h4>
<p>soft_unstable seems <code>deny</code> by default, but the error message is</p>
<div class="codehilite"><pre><span></span>error: use of unstable library feature &#39;test&#39;: `bench` is a part of custom test frameworks which are unstable
 --&gt; src/lib.rs:384:7
  |
9 |     #[bench]
  |       ^^^^^
  |
  = warning: this was previously accepted by the compiler but is being phased out; it will become a hard error in a future release!
  = note: for more information, see issue #64266 &lt;https://github.com/rust-lang/rust/issues/64266&gt;
</pre></div>


<p>so maybe it's not a rustdoc-rustc miscommunication, and just the message confusing me</p>



<a name="177327088"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177327088" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> lqd <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177327088">(Oct 04 2019 at 10:31)</a>:</h4>
<p>let me know if we need to do more testing</p>



<a name="177332205"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332205" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> pnkfelix <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332205">(Oct 04 2019 at 12:06)</a>:</h4>
<p>maybe I misunderstood what happened here</p>



<a name="177332244"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332244" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> pnkfelix <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332244">(Oct 04 2019 at 12:07)</a>:</h4>
<p>I had thought the issue was now hypothesized to be: There was no actual changes to resolution (we think) -- instead, a change to the doc-comment parser used by rustdoc caused it to start picking up tests that it wasn't picking up before,</p>



<a name="177332246"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332246" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> pnkfelix <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332246">(Oct 04 2019 at 12:07)</a>:</h4>
<p>and some of those tests were buggy</p>



<a name="177332261"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332261" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> pnkfelix <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332261">(Oct 04 2019 at 12:07)</a>:</h4>
<p>but those buggy tests were skipped (and thus silently accepted) in the past by crater</p>



<a name="177332314"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332314" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> pnkfelix <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332314">(Oct 04 2019 at 12:08)</a>:</h4>
<p>which means I don't think there's any follow-up work for the rustc team to do. The only questions I could imagine are whether the rustdoc team wants/needs to do anything to "address" this</p>



<a name="177332328"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332328" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> pnkfelix <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332328">(Oct 04 2019 at 12:08)</a>:</h4>
<p>(I suspect the answer at this point is "there's no value in going back to the old parser, or even trying to warning-cycle this change" -- but its not my decision to make, I think.)</p>



<a name="177332330"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332330" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> lqd <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332330">(Oct 04 2019 at 12:08)</a>:</h4>
<p>there are many crates in that issue and that describes most of them yes, however some of the code which was ignored and unignored seemed to showcase rustc issues, which seemed either known, or have since been fixed</p>



<a name="177332339"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332339" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> pnkfelix <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332339">(Oct 04 2019 at 12:09)</a>:</h4>
<p>hmm</p>



<a name="177332359"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332359" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> lqd <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332359">(Oct 04 2019 at 12:09)</a>:</h4>
<p>the other issue which could be a duplicate, and mentions the new test detection behaviour also mentions what you say, unlikely to rollback as it fixed other issues, but I don't know</p>



<a name="177332368"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332368" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> pnkfelix <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332368">(Oct 04 2019 at 12:09)</a>:</h4>
<p>If it seems to you that all of the rustc issues that have been identified this way are all already known and/or fixed</p>



<a name="177332423"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332423" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> pnkfelix <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332423">(Oct 04 2019 at 12:10)</a>:</h4>
<p>then I still don't think there's any follow-up work for rustc team?</p>



<a name="177332432"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332432" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> pnkfelix <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332432">(Oct 04 2019 at 12:10)</a>:</h4>
<p>Or is your point that you want to know how we can determine</p>



<a name="177332447"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332447" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> pnkfelix <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332447">(Oct 04 2019 at 12:10)</a>:</h4>
<p>if we need to look more deeply or not?</p>



<a name="177332451"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332451" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> lqd <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332451">(Oct 04 2019 at 12:10)</a>:</h4>
<p>I _could_ bisect those if you want, but since they seem fixed I'm not sure we need to</p>



<a name="177332456"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332456" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> pnkfelix <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332456">(Oct 04 2019 at 12:10)</a>:</h4>
<p>yeah I don't think we need to do that at this point</p>



<a name="177332461"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332461" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> pnkfelix <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332461">(Oct 04 2019 at 12:11)</a>:</h4>
<p>Now I understand your Q</p>



<a name="177332467"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332467" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> lqd <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332467">(Oct 04 2019 at 12:11)</a>:</h4>
<p>that is, I expect bisection to point to these known fixed issues</p>



<a name="177332489"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332489" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> lqd <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332489">(Oct 04 2019 at 12:11)</a>:</h4>
<p>the only new information I had was discovering a deny lint had a lint-like message and that's it</p>



<a name="177332506"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332506" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> pnkfelix <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332506">(Oct 04 2019 at 12:11)</a>:</h4>
<p>Right. The main benefit I could see from bisection would be to double check if the problems being fixed here are actually covered by whatever tests were added when the relevant bugs were fixed.</p>



<a name="177332553"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332553" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> pnkfelix <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332553">(Oct 04 2019 at 12:12)</a>:</h4>
<p>(as in, there <em>might</em> be E-needs-test cases here)</p>



<a name="177332560"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332560" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> pnkfelix <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332560">(Oct 04 2019 at 12:12)</a>:</h4>
<p>but I don't think its worth the effort</p>



<a name="177332571"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332571" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> pnkfelix <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332571">(Oct 04 2019 at 12:12)</a>:</h4>
<p>we have got a lot of other bugs that I think deserve attention</p>



<a name="177332573"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332573" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> lqd <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332573">(Oct 04 2019 at 12:12)</a>:</h4>
<p>alright, I agree :)</p>



<a name="177332673"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177332673" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> lqd <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177332673">(Oct 04 2019 at 12:14)</a>:</h4>
<p>I think we can close this one then, the "rustdoc changing behaviour" problem is probably for the better and "tracked" in another T-rustdoc issue <a href="https://github.com/rust-lang/rust/issues/61562" target="_blank" title="https://github.com/rust-lang/rust/issues/61562">#61562</a></p>



<a name="177333033"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177333033" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> pnkfelix <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177333033">(Oct 04 2019 at 12:20)</a>:</h4>
<p>sounds good to me</p>



<a name="177333585"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/rustdoc%20bisection%20%2361560/near/177333585" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> lqd <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/rustdoc.20bisection.20.2361560.html#177333585">(Oct 04 2019 at 12:29)</a>:</h4>
<p>done</p>



<hr><p>Last updated: Aug 07 2021 at 22:04 UTC</p>
</html>